fix(extensions): show extension ID in list output#1843
Conversation
There was a problem hiding this comment.
Pull request overview
Improves the UX of extension management by making the installed extension ID discoverable directly in specify extension list, aligning the list output with guidance shown in ambiguous-name error flows.
Changes:
- Print each installed extension’s
idon a separate (dim) line beneath the extension name inspecify extension list.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Display the extension ID below the name in `specify extension list` output. This allows users to easily copy the ID when disambiguation is needed. Fixes github#1832 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
mbachorik
force-pushed
the
fix/extension-list-show-id
branch
from
d566214 to
71ef55d
Compare
There was a problem hiding this comment.
Pull request overview
Updates the specify extension list CLI output to include each installed extension’s ID, making it easier for users to act on extensions when names are ambiguous (as suggested by existing CLI error guidance).
Changes:
- Print the installed extension
idon a dimmed line directly under the extension name inspecify extension list. - Add a CLI integration test asserting the list output includes the extension ID (plus name/version).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
src/specify_cli/__init__.py |
Adds a new output line in extension_list() to print ext["id"] beneath the extension name. |
tests/test_extensions.py |
Adds TestExtensionListCLI::test_list_shows_extension_id to validate the new list output includes the extension ID. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
There was a problem hiding this comment.
Pull request overview
This PR improves the usability of the extensions CLI by displaying each installed extension’s ID in specify extension list, aligning the output with guidance users receive when extension names are ambiguous (Fixes #1832).
Changes:
- Print the installed extension ID (dimmed) on its own line directly beneath the extension name/version in
specify extension list. - Add a CLI test asserting that
extension listoutput includes the extension ID (along with name and version).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
src/specify_cli/__init__.py |
Updates extension_list() output to include ext["id"] in the printed details for installed extensions. |
tests/test_extensions.py |
Adds an integration-style CLI test verifying specify extension list shows the extension ID. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Thank you! |
Display the extension ID below the name in `specify extension list` output. This allows users to easily copy the ID when disambiguation is needed. Fixes github#1832 Co-authored-by: iamaeroplane <michal.bachorik@gmail.com> Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
Summary
specify extension listoutputProblem
When extension names are ambiguous, users are told to use the extension ID:
But
specify extension listdidn't show the ID, forcing users to inspect.specify/extensions/registry.jsonmanually.Solution
Add the ID on a separate line (dim style) below the extension name:
Test plan
TestExtensionListCLI::test_list_shows_extension_idtestFixes #1832
🤖 Generated with Claude Code